Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

fix: temporarily removing onboarding wizard for updates #1273

Merged
merged 2 commits into from
Dec 19, 2023

Conversation

AvineshTripathi
Copy link
Collaborator

This PR temporarliy sets down the onboaring wizard as the backend support of the feature is still incomplete.

Some changes on the bakend side is done in this PR but since the main functionality will take some time to be figured out it would be better to keep the feature down till then.

Problem: When a cloud in onboarded the only thing the backend does currently is updating the db however scanning the account requires creation of clients and configs which is not possible with the current architecture.

@Azanul
Copy link
Collaborator

Azanul commented Dec 13, 2023

What is Source field added to all config structs?

@AvineshTripathi
Copy link
Collaborator Author

To create the config.toml file, for Aws sources can be env_var and so all the fields that were being sent from frontend I included that

@AvineshTripathi AvineshTripathi mentioned this pull request Dec 17, 2023
4 tasks
@Azanul
Copy link
Collaborator

Azanul commented Dec 18, 2023

@AvineshTripathi Lets only deactivate the wizard and remove everything other than that. We'll look back again after refactoring.

@AvineshTripathi
Copy link
Collaborator Author

@Azanul this should be good to go

Copy link
Collaborator

@Azanul Azanul left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

Copy link
Contributor

@jakepage91 jakepage91 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

GTG

@jakepage91 jakepage91 merged commit 07b65d3 into tailwarden:develop Dec 19, 2023
3 checks passed
@AvineshTripathi AvineshTripathi deleted the onboarding-bug branch August 31, 2024 15:07
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants